Skip to content

Conversation

fabpot
Copy link
Member

@fabpot fabpot commented Jul 20, 2025

Q A
Bug fix? yes
New feature? no
Docs? no
Issues
License MIT

Updated the code to the current names.

@fabpot fabpot force-pushed the demo-chatbot-fix branch from e6c8066 to 15e93ea Compare July 20, 2025 07:57
@chr-hertel
Copy link
Member

That shouldn't be needed anymore, we merged #154 already

@fabpot
Copy link
Member Author

fabpot commented Jul 20, 2025

That's because there is a composer.lock file in the demo directory. I suggest that we remove it until we have a first release. WDYT?

@chr-hertel
Copy link
Member

Ah, yes, I feel like that demo doesn't play well right now with the mono repo. removing the composer.lock might help 👍

@chr-hertel
Copy link
Member

And maybe switching to Symfony CLI as a next step for the PHP runtime

@fabpot
Copy link
Member Author

fabpot commented Jul 20, 2025

And maybe switching to Symfony CLI as a next step for the PHP runtime

I can work on it :)

What about using PostgreSQL for the vector DB? That would allow for a nice and simple way for developers to get started.

@chr-hertel
Copy link
Member

Postgres would easily integrate with standard DB tooling, yea, i like that - and since we have that bridge now, yes please :)

@fabpot
Copy link
Member Author

fabpot commented Jul 20, 2025

See #170 for a better fix.

@fabpot fabpot closed this Jul 20, 2025
chr-hertel added a commit that referenced this pull request Jul 20, 2025
This PR was merged into the main branch.

Discussion
----------

Remove composer.lock until we have a first release

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Docs?         | no <!-- required for new features -->
| Issues        | Closes #169
| License       | MIT

Until we have a release, it's better to always get the latest version of the code.

Commits
-------

6b4ed7f Remove composer.lock until we have a first release
@chr-hertel chr-hertel added the Demo Issues & PRs about the demo application label Jul 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Demo Issues & PRs about the demo application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants